Skip to content

Delete deployments from the Deployments view#83

Open
hauptmedia wants to merge 1 commit into
mainfrom
codex/deployment-delete
Open

Delete deployments from the Deployments view#83
hauptmedia wants to merge 1 commit into
mainfrom
codex/deployment-delete

Conversation

@hauptmedia

Copy link
Copy Markdown

Summary

  • add a delete action for the selected deployment in the Deployments view
  • confirm before deleting and call the existing deployment delete API with cascade=true
  • clear deployment resource state and route back to /deployments after deletion

Reference

Verification

  • node -e 'const fs=require("fs"); for (const locale of fs.readdirSync("public/locales")) JSON.parse(fs.readFileSync("public/locales/"+locale+"/translation.json", "utf8")); console.log("locale JSON ok")'
  • npx eslint src/pages/Deployments.jsx src/pages/Deployments.test.jsx
  • npx vitest run src/pages/Deployments.test.jsx
  • npx vitest run --maxWorkers=4
  • npm run build
  • Browser smoke at http://127.0.0.1:5175/deployments/dep1

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant